Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

useAccessibilityInfo update #174

Merged

Conversation

adkenyon
Copy link
Contributor

@adkenyon adkenyon commented May 14, 2020

Summary

I added several other Accessibility features to theuseAccessibilityInfo hook.

Note: this does break backwards compatibility with 0.59 which only implements two of the listed accessibility features

Test Plan

Tested the branch in our RN App

Compatibility

OS Implemented
iOS
Android
Web

Checklist

  • I have tested this on a device and a simulator
  • added the documentation in README.md
  • I updated the typed files (TS and Flow)
  • I've created a snack to demonstrate the changes

Unclear how to make a snack that points to a forked branch. Would love advice on this if expo snack supports it

@adkenyon adkenyon force-pushed the useAccessibilityInfo-update branch from 22a94da to 9d3ba97 Compare May 14, 2020 20:52
package.json Outdated Show resolved Hide resolved
@adkenyon adkenyon requested a review from LinusU May 18, 2020 01:21
src/useAccessibilityInfo.ts Outdated Show resolved Hide resolved
@adkenyon adkenyon requested a review from LinusU May 25, 2020 23:49
@adkenyon adkenyon force-pushed the useAccessibilityInfo-update branch from f022bbf to 7842610 Compare May 25, 2020 23:52
src/useAccessibilityInfo.ts Outdated Show resolved Hide resolved
src/useAccessibilityInfo.ts Outdated Show resolved Hide resolved
adkenyon and others added 3 commits May 26, 2020 10:00
Co-authored-by: Linus Unnebäck <linus@folkdatorn.se>
Copy link
Member

@LinusU LinusU left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great 👍

@LinusU LinusU added enhancement New feature or request minor Increment the minor version when merged release Create a release when this pr is merged labels May 27, 2020
@adkenyon
Copy link
Contributor Author

adkenyon commented Jun 4, 2020

Thanks @LinusU! anything else you need me to do?

@LinusU
Copy link
Member

LinusU commented Jun 5, 2020

Nope, sorry for the delay, merging and releasing now!

Thank you so much for the contribution!! 🙌

@LinusU LinusU merged commit 8bc9999 into react-native-community:master Jun 5, 2020
@pvinis
Copy link
Member

pvinis commented Jun 5, 2020

🚀 PR was released in v2.6.0 🚀

@pvinis pvinis added the released This issue/pull request has been released. label Jun 5, 2020
@adkenyon adkenyon deleted the useAccessibilityInfo-update branch June 5, 2020 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request minor Increment the minor version when merged release Create a release when this pr is merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants